-
Notifications
You must be signed in to change notification settings - Fork 3
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
159 create an action to publish to pypi when we make a release #184
Merged
K20shores
merged 17 commits into
main
from
159-create-an-action-to-publish-to-pypi-when-we-make-a-release
Aug 7, 2024
Merged
159 create an action to publish to pypi when we make a release #184
K20shores
merged 17 commits into
main
from
159-create-an-action-to-publish-to-pypi-when-we-make-a-release
Aug 7, 2024
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
WardF
approved these changes
Aug 5, 2024
boulderdaze
approved these changes
Aug 5, 2024
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nice!
mattldawson
approved these changes
Aug 6, 2024
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
nice!
K20shores
deleted the
159-create-an-action-to-publish-to-pypi-when-we-make-a-release
branch
August 7, 2024 13:16
K20shores
added a commit
that referenced
this pull request
Aug 7, 2024
* Start on test_micm_box_model.F90. * Start on test_micm_box_model.F90. * In test_micm_box_model, create micm_t solver. * Initialize some variables. * Use configs/analytical. * Use configs/analytical. * Added micm solve. * Start on Chapter 2. * Literal include for test_micm_box_model. * Literal include json configs. * Updated CMakeLists.txt. * Add config file description. * Tutorial Chapter 2. * Tutorial Chapter 2. * Minor formatting. * Minor formatting. * Added link to MICM docs, test program output. * Start on tutorial CMakeLists.txt. * Created MUSICA/tutorial subdir. * Added MUSICA_INCLUDE_DIR and MUSICA_LIBRARY_DIR. * Switch to MUSICA_INSTALL_DIR/include /lib. * Include musicaConfig.cmake. * Include addtional cmake files. * Added Fortran to CMakeList LANGUAGES. * Added target_link_directories and libraries. * Added find_package(musica). * Comment use_package(musica_fortran), not working. * Added cmake usage comments. * Restored find_package for musica and musica_fortran. * Start on Tutorial Chapter 0 text. * Added literal include of Ch 0 CMakeLists.txt. * Chapter 0 intro. * Tutorial Chapter 0. * update cmake options * set vector matrix dimension * Use music-fortran instead of musica_fortran. * Added test_micm_box_model executable. * Updated test_micm_box_model with new micm solve. * Added deallocate( micm ) in test_micm_box_model. * Auto-format code using Clang-Format (#170) Co-authored-by: GitHub Actions <actions@github.com> * pypi version badge * fair software checklist badge * compilation flag for grid cells, vector matrix dimension * c api for vector ordered matrix * add vector matrix test * update fortran api * chage the build type * update python wrapper * resolve memory leak * fix intel docker * tuvx on * micm git tag * edit erorr * opdate the option name for open mp * remove open mp fortran in c tests * edit comment * Removed find_package lines in this version. * replace unique ptr T to T * replace tempalte with auto * Add profiles for TUV-x (#171) * add profiles for TUV-x * update TUV-x commit * split tuvx source into separate headers and modules * split TUV-x interface into separate modules * allow grid creation * fix grid updates * update fortran tuvx wrapper * address cross-compiler issue * fix invalid free * update actions * fix grid map ownership problem * finish profile and profile_map * address review comments * Auto-format code using Clang-Format * add map header * expose solver type enum to python * back to template * back to template * enum for solver type * enum solver type for fortran * add fortran 2023 feature comment * pass num grid cells to constructor * code clean up * space uniformly * specify enabled languages in top level cmake * comment out the tuvx openmp tests * revert back * revert back * Added tutorial/demo.f90 as a unit test. * Auto-format code using Clang-Format * Set demo.f90 path with PROJECT_SOURCE_DIR. * Added tutorial dir .dockerignore. * add radiator wrapper * working-on file of radiator header * fix typos, and incomplete parts * code clean up * add radiator src * fix bugs * fix bugs * Moved tutorial subdir to fortran/test/tutorial. * Removed demo_c executable. * Updated docs. * fix bugs in c sources * fix bugs in f90 sources * fix bugs * temp add radiator test * fix bugs * passed test * incomplete test * temp file for transfer * working version * fix test * fix bugs * fix fortran bugs * add radiator fortran test * test tag * add fortran test * add tests back: * fix docker integration * revert cmake * clean up * code cleanup * code clean up * revert back to main branch * 159 create an action to publish to pypi when we make a release (#184) * testing automated pypi release * trying to appropriately name the wheels * only need one version of python to build wheels for all other versions * trying to get macos to build successfully * excluding universal2 arch build for macos * seeing if arm64 works * trying to skip musllinux * trying to choose linux architecture support * trying to skip musllinux again * adding intel based mac build * trying to set the osx deployment target * trying a different way * that wasn't it * trying to force a minimum of 10.15 for macos? * setting action trigger * undoing version change * correcting project name * Address review comments * fix tuvx git tag * Python error checking (#189) * adding error checking, some debug statements for the python wrapper * adding error checking to python, updating micm tag * getting tests to pass * correcting a fortran test * newest tuvx commit * I don't know what's wrong with nvidia * Updating citation version number --------- Co-authored-by: David Fillmore <fillmore@ucar.edu> Co-authored-by: Jiwon Gim <jiwongim@ucar.edu> Co-authored-by: David Fillmore <1524012+dwfncar@users.noreply.github.com> Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com> Co-authored-by: GitHub Actions <actions@github.com> Co-authored-by: Matt Dawson <mattldawson@gmail.com> Co-authored-by: Jiwon Gim <55209567+boulderdaze@users.noreply.github.com>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Closes #159
Proof that it works, but these will be deleted. Successful CI run that built the most recent of these